[llvm-commits] [llvm] r124720 - in /llvm/trunk: include/llvm/ADT/Triple.h lib/Support/Triple.cpp unittests/ADT/TripleTest.cpp

[Renato Golin]

> Remove NoVendor and NoOS, added in commit 123990, from Triple.  While it
> may be useful to understand "none", this is not the place for it.  Tweak
> the fix to Normalize while there: the fix added in 123990 works correctly,
> but I like this way better.  Finally, now that Triple understands some
> non-trivial environment values, teach the unittests about them.

Thanks Duncan!

As discussed, that seem to fix the problems. I'll add later some of the triples we discussed to the normalization unit tests, just to make sure we covered all cases.

I'll also re-run all tests with my Clang patch, and probably add some more logic to Clang (like don't disable fpelim in EABI mode) and other stuff.

best,
--renato

-- IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium.  Thank you.