[PATCH] D112463: [LangRef][NFC] Remove incorrect vector alignment rules

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 25 09:20:29 PDT 2021


aeubanks added a comment.

it'd be good for somebody with more background to confirm if this is intended or not



================
Comment at: llvm/unittests/IR/DataLayoutTest.cpp:102
+  // The alignment for a vector type larger than any specified vector type uses
+  // the natual alignment as a fallback. This is contrary to old LangRef
+  // documentation which claimed it would use the largest vector type smaller
----------------
natural


================
Comment at: llvm/unittests/IR/DataLayoutTest.cpp:102
+  // The alignment for a vector type larger than any specified vector type uses
+  // the natual alignment as a fallback. This is contrary to old LangRef
+  // documentation which claimed it would use the largest vector type smaller
----------------
aeubanks wrote:
> natural
I would just delete everything after this, it's not useful to anybody in the future.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112463/new/

https://reviews.llvm.org/D112463



More information about the llvm-commits mailing list