[PATCH] D44559: [Sema] Wrong width of result of mul operation

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 16 11:36:43 PDT 2018

lebedev.ri added inline comments.

Comment at: lib/Sema/SemaChecking.cpp:8541
+      switch (C.getTargetInfo().getTriple().getArch()) {
+      case llvm::Triple::x86:
+      case llvm::Triple::x86_64: {
I agree with @RKSimon, i don't see why this would be arch-specific.
If this is correct, maybe add explanation as a code comment?

  rC Clang


More information about the cfe-commits mailing list