Hi Hal, Thanks for the review! No worries about the delay.
The diagnostic handling code doesn’t use isa or dyn_cast, rather it uses switches to select between different types. I modified the patch to use a subclass rather than a member variable. Let me know what patch you think would work out better?
Using -O1 with -fvectorize seems to work, at least it is a smaller set of passes than O3. I attached the updated patches. I also noticed that DiagnosticInfoOptimizationBase::classof() was incorrectly implemented. It would need its own diagnostic kind, but that doesn’t make sense because you would never instantiate the base class. I thought it was best just to remove it. See the third patch. Tyler |
LLVM-Late-evaluation-of-vectorization-requirements.patch
Description: Binary data
Clang-Add-diagnostic-to-append-fp-commute-clang-options-to.patch
Description: Binary data
LLVM-Removed-unused-and-broken-classoff-method-in-base-cl.patch
Description: Binary data
_______________________________________________ cfe-commits mailing list cfe-commits@cs.uiuc.edu http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits