[PATCH] D41742: [MSF] Fix FPM interval calculation

[Colden Cullen via Phabricator via llvm-commits]

colden added inline comments.


================
Comment at: llvm/include/llvm/DebugInfo/MSF/MSFCommon.h:118
+    // the number of fpm blocks.
+    int N = AltFpm ? 2 : 1;
+    return divideCeil(NumBlocks - N, BlockSize);
----------------
colden wrote:
> Naming nitpick: In other contexts, `AltFpm` implies the opposite of `Msf.SB->FreeBlockMapBlock` (i.e. `3 - FpmBlock`). How about instead of taking that param here, you take an int for `FpmIndex` (or something), require it be 1 or 2, and then in the `MSFLayout` overload you can do the `AltFpm` transformation?
Follow up question: shouldn't it not matter which FPM you're using?


https://reviews.llvm.org/D41742